-
Notifications
You must be signed in to change notification settings - Fork 450
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(docs): add examples for ucloud provider (ts/python/java/c#/go) #790
Conversation
274de12
to
65e06d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks a ton for sending in a PR 👍
6a19365
to
0d9ecf4
Compare
I'm sorry for the two failures that appeared. 😂 I have tried to fix one of them, but there are some errors when I run the task examples:integration:typescript and integration:windows:java on local with $ yarn integration:windows:java
run-against-dist.bat: command not found $ yarn examples:integration:typescript
silly fetchPackageMetaData error for @types/node@* Unexpected end of JSON input while parsing near '...vln4sHRLF19hjFZeQ0LFd' How can I execute them on my desktop? Thanks. 😄 |
Running on Windows currently requires installing all of the tools for each language you are testing with. Looks like you are running correctly within the container. Could have been a temporary issue or perhaps some network configuration issue on your machine. |
Hi @DanielMSchmidt, Thanks for @jsteinich, the integration test In this log details, this is a timeout error, could you approval to continue the workflow, please. thank you ~ 😁 |
I had passed the But after
Can you help me to merge this pull request? What should I do for the next step? Thanks a lot 😄 @DanielMSchmidt |
It seems like the current test could be improved as it seems to expect this repository to be checked out into a directory called I'm going to post a suggestion via a review which should fix this for you. edit 1: I just noticed that I cannot add suggestions on files that weren't changed 😅 Going to add a separate PR so you can either cherry-pick or rebase/merge main as soon as it get's merged edit 2: Link to PR |
don't require the terraform-cdk to be checked out into a directory with exactly that name for this test case to succeed. Reported by @yufeiminds in https://github.com/hashicorp/terraform-cdk/pull/790\#issuecomment-879046497
Thanks to @ansgarm. I have passed the |
Hello, thanks for your help. Can this PR be merged? @DanielMSchmidt @jsteinich @ansgarm |
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Add examples for UCloud provider.
Languages
Additional Infomation